This could also be seen from the lack of coverage on CodeCov as these are impossible branches to reach.

And have a linear code flow that is easier to reason

…hive()

We don't need to be assigning the realpath to fname, we can just use it directly.
This also makes the efree(my_realpath) call not seem completely random.

It serves no purpose as fname is never modified so there is nothing to restore.
Just use fname directly instead.

Just check in advance if we don't have an archive to checkin in the manifest cache.

…_archive()

We don't actually care about the returned pointer, so use a clearer API to better communicate intent.

The naming between fd and fd_ptr seem to indicate one is not a pointer, but they both are.
Moreover, there is no reason to copy the same value to the variables at the same time.